-
Notifications
You must be signed in to change notification settings - Fork 18
Fixes #58 Parametrize irc-channel in process VCSCOMMIT_NOTIFY #59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
Where would this log_channel parameter come from? AFAIK only the WebhookMapping fields and the msg is passed to the process when launching. I think it might be better if you just modify the process definition in you deployment and set what ever channel you want there. It is really simple process and adding configuration options just makes it more complicated. Currently the process definitions are not marked as configuration files in the RPM, so you need to copy them somewhere else and point the options in webhooks.conf [processes] section to correct location, so that your modifications are not overwritten when updating. |
|
For participants in boss-launcher-webhook area, it will be great to have ability to send notifications by simple calling: I want to send important notification from participant to irc, i can make a separate process, but for another participant new process will be needed, it is not a good idea. If there is no security issue |
|
It seems there are a few things going on :) My main concern here is that participants should (almost) never launch new processes or call other participants. Tracing process flows is hard enough without launching new processes just to send messages to irc. The purpose of a process is to define a flow of execution and how to handle participant responses (email irc etc) so having the participant decide this is a bit dubious. If a participant needs to contact the irc channel then it should call the web service directly. Ideally this would be done via a simple library. So because that's the reason for the change then I would say no. If this was part of a bigger change that collected webhook configuration into one place then that may make sense - but I'd object to any participant that then happened to use the launch_notify() 'private' webhook method ;) |
|
Thank you, i got it. |
|
There are "variables" parameter in RuoteAMQP.launcher.Launcher#launch But couldn't find out how to use that variables in pdef process, any idea? |
Accept 'log_channel' as parameter
When parameter is not passed use default value